-
-
Notifications
You must be signed in to change notification settings - Fork 812
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
assure function #1325
assure function #1325
Conversation
Would like a few more examples showing the use of It may even be useful to have a few |
@fubuloubu I am not that keen that we have 2 different asserts, and I still don't think having two types is a good idea (even more so because we have reason string support -> e.g. |
Having thought about it some more. Scratch most of what I said above. I am actually OK with having the function, especially because of the tool support (invalid being the de facto). |
Okay, I assigned myself to add documentation on this feature and how to use it effectively. Let's keep #711 open until that is complete. |
Cool, thanks :) it's been a while since I have thought about the analyzers, do we still have that model checker issue somewhere? |
It would be fantastic to get someone to review this new feature. cc @maurelian @wuestholz @japesinator @montyly could you aid us? |
@fubuloubu @jacqueswww Great to see this being added! In general, any solution that allows to distinguish between parameter validation ( The ideal solution would probably be to change the semantics of the existing On the other hand, I understand that you might not want to change the semantics and then your solution will work as long as it is well-documented. |
@fubuloubu @wuestholz the main reason we didn't want to change |
Meeting notes: closing in favour a proposal to be done where |
@maurelian @wuestholz @japesinator @montyly we've made a decision to modify the syntax to what @jacqueswww has proposed here: #711 (comment) Please give us feedback! |
What I did
Fixes #711.
How I did it
How to verify it
Play with the assure function.
Description for the changelog
Added assure function.
Cute Animal Picture